Simplify setStatus implementation in run. NFC#27121
Conversation
|
Pretty much all the code size tests get worse. Are we not testing with |
A lot of the codesize tests do use For example all of the |
sbc100
left a comment
sbc100
left a comment
There was a problem hiding this comment.
More code simplification... way nicer now. But still sadly a codesize regression :(
sbc100
left a comment
sbc100
left a comment
There was a problem hiding this comment.
Simplified again.. now the regression is only 11 bytes compressed.
sbc100
left a comment
sbc100
left a comment
There was a problem hiding this comment.
Simplified more.. can't quite remove the regression though.
The codesize costs here only apply to builds that include `setStatus` in `INCOMING_MODULE_JS_API`, so don't apply to builds that really care about code size (i.e. when `-sINCOMING_MODULE_JS_API=[]` is used).
2 participants
Note: The codesize costs here only apply to builds that include
setStatusinINCOMING_MODULE_JS_API, so don't apply to builds that really care about code size (i.e. when-sINCOMING_MODULE_JS_API=[]is used).